Skip to content

Conversation

@renedekat
Copy link
Contributor

This patch enables passing custom parameters like --region or -subdomain.

valet share --region=eu -subdomain=my_custom_domain

If you have a paid version of Ngrok the subdomain param is available, which is very handy when working with customers. Share the link once, instead of sharing a new random link every time you want your client to look at your progress.

This makes it possible to add custom parameters to ngrok. For example:
--region=eu
-subdomain=YOUR_SUBDOMAIN

Some features, like -subdomain, are only available with the paid version of Ngrok.
Added option to pass custom parameters to ngrok
@irazasyed
Copy link
Contributor

This is great. Should be merged. Have paid version of Ngrok and I was trying to get the sub-domain working! 👍

@adamwathan
Copy link
Contributor

Looks good, thanks!

@adamwathan adamwathan merged commit 435e7b9 into laravel:master Aug 31, 2016
@renedekat
Copy link
Contributor Author

@irazasyed You'll be pleased now :)

@irazasyed
Copy link
Contributor

@renedekat Yep! Thank you :)

@alfhen
Copy link

alfhen commented Oct 17, 2016

@adamwathan Got any estimate on when v1.1.23 will be released? Would be nice to have this feature 😄

@adamwathan
Copy link
Contributor

I would recommend just running master for now. There's a weird bug in
latest Caddy that makes you have to refresh the page multiple times to see
your changes and it just seems to happen randomly, so need to get that
resolved before tagging :/
On Mon, Oct 17, 2016 at 6:41 AM Alf notifications@github.com wrote:

@adamwathan https://github.com/adamwathan Got any estimate on when
v1.1.23 will be released? Would be nice to have this feature 😄


You are receiving this because you were mentioned.

Reply to this email directly, view it on GitHub
#112 (comment), or mute
the thread
https://github.com/notifications/unsubscribe-auth/AEH3bOKaGxrr9oJ9yi6xY0gy5vs2XzzLks5q01DfgaJpZM4Jj2vu
.

@sleicester
Copy link

Just wanted to change the region and stumbled across this, done. Thanks a lot 👍

drbyte added a commit to drbyte/valet that referenced this pull request Dec 6, 2019
This PR allows passing a specific domain name to `valet share` in order to have the app be served for that domain.
Now how it works is this:
- (NEW) if a domain name is passed, it checks whether it matches a link (which would be required for multiple domains served by same app project), and uses that
- if a domain name is NOT passed, it looks up any links for current directory, and uses the first found link
- else falls back to current project foldername

It also still allows passing through custom ngrok parameters if desired (ref: laravel#112), as either the 2nd parameter (no domain name passed) or (NEW) 3rd parameter

Fixes laravel#537
drbyte added a commit to drbyte/valet that referenced this pull request Dec 7, 2019
This PR allows passing a specific domain name to `valet share` in order to have the app be served for that domain.
Now how it works is this:
- (NEW) if a domain name is passed, it checks whether it matches a link (which would be required for multiple domains served by same app project), and uses that
- if a domain name is NOT passed, it looks up any links for current directory, and uses the first found link
- else falls back to current project foldername

It also still allows passing through custom ngrok parameters if desired (ref: laravel#112), as either the 2nd parameter (no domain name passed) or (NEW) 3rd parameter

Fixes laravel#537
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

6 participants